Skip to content

change #include <unzip.h> to #include <minizip/unzip.h>#274

Open
Architector4 wants to merge 4 commits intocortex-command-community:developmentfrom
Architector4:patch-minizip-import
Open

change #include <unzip.h> to #include <minizip/unzip.h>#274
Architector4 wants to merge 4 commits intocortex-command-community:developmentfrom
Architector4:patch-minizip-import

Conversation

@Architector4
Copy link
Copy Markdown
Contributor

This lets the codebase built on my Arch Linux system again. HeliumAnt said it should be fine to do this lmao

@Architector4
Copy link
Copy Markdown
Contributor Author

msbuild go shit yourself

i had a rough day ok

This should have been causing issues for a while, unknown why it hasn't
yet. The changes to minizip pkg-config that make this necessary are already 2y old and are already
part of the previous release tag.
HeliumAnt
HeliumAnt previously approved these changes Mar 5, 2026
Copy link
Copy Markdown
Contributor

@HeliumAnt HeliumAnt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lol, the only reason this wasn't already an issue 2y ago, is that arch skipped the 1.3.1.2 tag on zlib and minizip.

s20610 added a commit to s20610/Cortex-Command-Vibe-Wars that referenced this pull request Mar 29, 2026
build(minizip): use SYSTEM_MINIZIP header paths on linux/darwin (PR cortex-command-community#274)
@Causeless Causeless added this pull request to the merge queue Apr 2, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 2, 2026
@Architector4
Copy link
Copy Markdown
Contributor Author

Architector4 commented Apr 2, 2026

oh my lord lmao

edit:

/home/runner/work/Cortex-Command-Community-Project/Cortex-Command-Community-Project/github/actions/osxcross/osxcross/target/bin/../macports/pkgs/opt/local/include/minizip/ioapi.h:74:10: fatal error: ints.h: No such file or directory

what the FUCK

@Causeless Causeless added this pull request to the merge queue Apr 4, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 4, 2026
@Causeless
Copy link
Copy Markdown
Contributor

tf is going on with this :(

@HeliumAnt
Copy link
Copy Markdown
Contributor

macports has a broken version of the minizip package, there's already a fixed version of the package available (within hours of the broken release and already for a month), but it's somehow still using the broken version (the broken release has ~5 downloads this month and I think thats all ours)
I have no clue why.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants